Use GROUP consistently for sublibrary tests - #3909
Conversation
|
The three current |
|
Updated failure classification after more cells completed:
Thus the newly visible QA/downstream failures are substantive tests exposed by the selector repair, not evidence of another silent no-op. |
|
The clean-base DiffEqBase |
|
CI timer check (2026-07-15 09:49 EDT): the changed downstream Integration workflow run (29405724542) is terminal with 12/12 jobs successful. This confirms that the generic group values introduced by this PR are accepted and those downstream jobs actually execute. The repository-wide rollup is not terminal yet: 152 checks have succeeded, 12 have failed in unrelated monorepo downgrade/sublibrary suites, 3 were skipped, and 5 remain queued (four long StochasticDiffEq groups plus the GPU job). The completed failures are being reproduced separately on a clean ⎿ Setting 3-hour timer for CI completion as per CLAUDE.md instructions... |
|
Re-audited the current registered-package downstream workflow inventory on 2026-07-26. There are 23 active shared-workflow lanes targeting the four migrated runners covered here: DiffEqBase (12), DelayDiffEq (5), StochasticDiffEq (5), and DiffEqDevTools (1). Each currently exports only generic Fresh local verification from this PR head: |
723ee8f to
96008a4
Compare
|
Rebased onto current Local verification after rebase:
|
96008a4 to
7379cd7
Compare
|
Rebased onto current Local verification after rebase:
The prior CI’s seven red lanes all reached the test runner (one was canceled while in it); they are live test/infrastructure outcomes, not group-routing silence. |
7379cd7 to
134aa60
Compare
Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
134aa60 to
0aa6457
Compare
Summary
Why
SciML downstream workflows export GROUP. A separate OrdinaryDiffEq-specific group variable made direct sublibrary runs inconsistent and allowed downstream group requests to be ignored. There is now a single group-routing interface for all OrdinaryDiffEq sublibrary runners.
Ignore this PR until reviewed by @ChrisRackauckas.
Local verification